-
Notifications
You must be signed in to change notification settings - Fork 565
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Sweeping LV522 Changes + FORECON GL replaced by FORECON Sniper (Without the sniper) #3917
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
You currently have a negative Fix/Feature pull request delta of -45. Maintainers may close this PR at will. Fixing issues or improving the codebase will improve this score. |
github-actions
bot
added
Mapping
did you remember to save in tgm format?
Feature
Feature coder badge
labels
Jul 18, 2023
Nanu308
approved these changes
Jul 22, 2023
Nanu308
requested changes
Jul 22, 2023
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
e
realforest2001
changed the title
(Requesting tactical testmerge) Sweeping LV522 Chances + FORECON GL being replaced by FORECON Sniper (Without the sniper)
Sweeping LV522 Changes + FORECON GL replaced by FORECON Sniper (Without the sniper)
Jul 24, 2023
3 tasks
3 tasks
harryob
requested changes
Jul 28, 2023
harryob
approved these changes
Jul 28, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Feature
Feature coder badge
Mapping Approved
adds 500 new dict keys
Mapping
did you remember to save in tgm format?
Testmerge Candidate
we'll test this while you're asleep and the server has 10 players
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
About the pull request
This PR makes alot of small changes to LV522 along with alot of big ones mainly
Blocks off the area north of fitness to prevent a "just go north" meta
Adds a new more central way to the reactor that branches off to 4 flanks should give LZ2 more use since it's a bit more central than LZ1
Adds a new route into the reactor from the SE
Blocks off the small alleyway north of the engineering funnel people into the dorm building
tightens up some parts of the reactor loosens some other parts
removes some detailing in the main hive areas to help builders
Adds new secrets to be found on the map
adds an overextension flank for those very brave marines to use
Changes to LZ2 making it more compressed
Addition of the FORECON Sniper (replacing the FORECON GL)
the FORECON sniper is a unique take on FORECON he spawns away from his fellow survivors next to the corpse of his battle buddy the advantage given to the sniper however is his thermal tarp a subtype of the ghillie suit with a placeholder sprite at the moment that will allow the sniper to hide and stealthily move throughout the colony to hopefully regroup with his squad
the M42A rifle has been removed from the map
Adds a small piece of text explaining that FORECON should probably move west to the crashed dropship to the FORECON intro
swaps the black weedable turf with a browner one for people who have bad eyes and were mistaking it for weeds
swaps the assignments for defines bc nanu told me someone would ask me to do it at somepoint
gives FORECON radioheadsets with the SOC and common channel because the desc literally says FORECON use it and it doesn't make sense for them to be radio-less
Explain why it's good for the game
Map stuff:
LV522 currently just has a "go north" problem this PR aims to fix that by opening more routes for the marines to go, centralize the main route in an attempt to see more use in the rest of the colony and give LZ2 more use I'd really like to see this test-merged for a few days so see what needs addressing in this new area
the wall north of engineering looks kinda funky but having it there funnels people into the building and into the eastern section of the colony rather than a straight line to the reactor
The FORECON Sniper:
Currently, FORECON GL is kinda unfitting and also kinda sucks the sniper, on the other hand, would be a perfect fit for FORECON now I'm not going to pretend the sniper isn't kinda OP right now so the FORECON sniper won't actually get his M42A just the ghillie suit that gives him still some uniqueness as a survivor as well as an incentive to survive beyond the life of the regular sniper (to steal his gun)
Testing Photographs and Procedure
Screenshots & Videos
Put screenshots and videos here with an empty line between the screenshots and the
<details>
tags.Changelog
🆑 SpartanBobby
add: Added a new survivor to LV522 the FORECON sniper he spawns alone his only company being the corpse of his dead battle buddy, good luck
maptweak: Sweeping changes to LV522 including the reactor, north of engineering, LZ1, the entire central area of the map, and north of fitness in an attempt to see more of the colony used and to incentivize flanking
/:cl: